test(ai-gateway-provider): cover unified dynamic routes - #619
dhruvil-codes wants to merge 1 commit into
Conversation
|
ArshVermaGit
left a comment
ArshVermaGit
left a comment
There was a problem hiding this comment.
Nice test addition. Covering the unified dynamic route improves confidence in the AI Gateway routing flow and helps catch regressions in the affected path. The changes are focused and fit well with the existing test suite. LGTM.
2 participants
Summary
Adds live E2E coverage for AI Gateway unified dynamic routes.
The existing
ai-gateway-providerE2E suite covers provider/BYOK routing through the universal endpoint, but does not exercise a dynamic route resolving to a Workers AI model.This adds a gated E2E case using:
aigateway(unified(DYNAMIC_ROUTE))and verifies that
generateTextsuccessfully returns the expected response through the universal-endpoint REST path.Motivation
Issue #617 reports that a dynamic route targeting Workers AI fails through the AI Gateway universal endpoint while the same request succeeds through
/compat/chat/completions.This test exercises the affected
ai-gateway-providerpath and provides regression coverage for the universal-endpoint dynamic-route flow.Testing
pnpm --filter ai-gateway-provider type-checkvitest --config vitest.e2e.config.ts --watch=false test/e2e/gateway.e2e.test.tsoxfmt --check test/e2e/gateway.e2e.test.tsAll checks pass locally. The live dynamic-route case is gated by
CLOUDFLARE_AI_GATEWAY_DYNAMIC_ROUTE.Refs #617